-
Notifications
You must be signed in to change notification settings - Fork 144
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Docs] colab notebook for chart #618
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Definitely just remove the ipython lower bound. We should remove it anyway. For the contextvars error, first step would be to get a working Dash app on colab. This might require a couple of hacks like they do on py.cafe, but worth checking the plotly forums in case someone has done it before also. |
Putting all vizro related code in one cell works in terms of getting a dashboard build once without error. But if I just rerun the same cell (contains all vizro related code) the Lookup error happens again. to replicate in a colab notebook
This cell can run successfully once. Rerun doesn't work
|
…docs/add_colab_link
Overall I think the experience in Colab shows why I disliked it in the past :) (which is no critique of this work!!), and we really should have a UI Things I dislike and maybe we can fix it a little (with comments, explanations or slight adjustments):
Actual notebook flow: While I like the model dropdown, and data upload, I think the user journey is different for most cases, ie we should maybe tailor it a little more to the below:
|
Tried many methods but didn't find a feasible way to put run button (a ipywidgets) and figure.show() into once cell output, these 2 are conflicting, which causes "figure.show()" creates only an empty space in output. In the end, I have to remove the run button. Please try it out one more time, see if the user experience improves a bit 🙏 |
I like it a lot more now! I think we are basically there - maybe we could add a note to the two cells that might be rerun (the data one and the create chart one) that hints users to click the run button. But I think then that's all, good job! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, this will facilitate more users for sure!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Yay, now it's a much better user experience 🚀
vizro-ai/changelog.d/20240807_005926_lingyi_zhang_add_colab_link.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. 🚀
Description
Add colab integration to enable easy access to Vizro-AI.
This PR now is for chart creation notebook only
Click "View file" to use the colab link
Pending issues:
TODOs before final merge:
Screenshot
Notice
I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":